Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix: api not getting called due to user permission data not updated #1565

Merged
merged 2 commits into from
Oct 7, 2024
Merged

fix: api not getting called due to user permission data not updated #1565

merged 2 commits into from
Oct 7, 2024

Conversation

gitanjli525
Copy link
Contributor

@gitanjli525 gitanjli525 commented Oct 7, 2024 •
edited
Loading

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

userpermission data not getting set

Motivation and Context

bugfix

How did you test it?

connectors api is being called on refresh

Where to test it?

  • INTEG
  • SANDBOX
  • PROD

Checklist

  • I ran npm run re:build
  • I reviewed submitted code
  • I added unit tests for my changes where possible

@gitanjli525 gitanjli525 self-assigned this Oct 7, 2024
@gitanjli525 gitanjli525 requested a review from a team as a code owner October 7, 2024 08:57
@semanticdiff-com SemanticDiff.com
Copy link

Review changes with SemanticDiff.

@gitanjli525 gitanjli525 added S-waiting-on-review Status: This PR has been implemented and needs to be reviewed core Status: issues related to core labels Oct 7, 2024
@gitanjli525 gitanjli525 linked an issue Oct 7, 2024 that may be closed by this pull request
[FIX]: add delay as hotfix for setting userPermissionJson #1568
Closed
@prajwalnl0 prajwalnl0 added this pull request to the merge queue Oct 7, 2024
Merged via the queue into main with commit ad7d230 Oct 7, 2024
11 of 12 checks passed
@prajwalnl0 prajwalnl0 deleted the add-delay branch October 7, 2024 10:54
@github-actions github-actions bot added Closed Applied to issues or pull requests that have been successfully resolved or completed and removed S-waiting-on-review Status: This PR has been implemented and needs to be reviewed core Status: issues related to core labels Oct 7, 2024
@prajwalnl0 prajwalnl0 mentioned this pull request Oct 7, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JeevaRamu0104 JeevaRamu0104 JeevaRamu0104 approved these changes

@sagarnaikjuspay sagarnaikjuspay sagarnaikjuspay approved these changes

Assignees

@gitanjli525 gitanjli525

Labels
Closed Applied to issues or pull requests that have been successfully resolved or completed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FIX]: add delay as hotfix for setting userPermissionJson
4 participants
@gitanjli525 @JeevaRamu0104 @sagarnaikjuspay @prajwalnl0